Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix SSblackbox and SSticker for terror spiders #1686

Merged
merged 3 commits into from
Dec 8, 2024

Conversation

Fullonibus
Copy link

@Fullonibus Fullonibus commented Dec 4, 2024

Что этот PR делает

Был рефакторинг биоугроз у офов, подтягивает актуальность модульных паучков до него.

Почему это хорошо для игры

Не будет ломаться логика работы с тикером и биоугрозами.

Изображения изменений

Тестирование

Локально проверил затронутые проки.

modular_ss220/events/code/SSticker.dm Outdated Show resolved Hide resolved
modular_ss220/events/code/SSticker.dm Outdated Show resolved Hide resolved
modular_ss220/events/code/SSticker.dm Outdated Show resolved Hide resolved
modular_ss220/events/code/spider_terror.dm Outdated Show resolved Hide resolved
@m-dzianishchyts m-dzianishchyts changed the title Fix SSticker for spiders Fix SSblackbox for terror spiders Dec 4, 2024
@m-dzianishchyts m-dzianishchyts changed the title Fix SSblackbox for terror spiders Fix SSblackbox and SSticker for terror spiders Dec 4, 2024
m-dzianishchyts
m-dzianishchyts previously approved these changes Dec 4, 2024
@m-dzianishchyts m-dzianishchyts dismissed their stale review December 4, 2024 18:38

еще в обсуждении

@m-dzianishchyts m-dzianishchyts added the Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. label Dec 4, 2024
@m-dzianishchyts m-dzianishchyts removed the Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. label Dec 7, 2024
@m-dzianishchyts m-dzianishchyts merged commit 8469390 into ss220club:master Dec 8, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants